Remove deprecated functionality#308
Open
syntron wants to merge 50 commits intoOpenModelica:masterfrom
Open
Conversation
d3d62bb to
f8f0b51
Compare
11dec82 to
144fa10
Compare
66a8d9b to
1587a59
Compare
1587a59 to
9039fb2
Compare
9039fb2 to
4263a08
Compare
Contributor
Author
|
@adeas31 this would be something for milestone 5.0.0 - one main release after the warning was introduced (= 4.0.0) |
672a250 to
d2a0d1c
Compare
d2a0d1c to
7cd44f1
Compare
2af0ab0 to
cf44464
Compare
cf44464 to
30c29b8
Compare
Pylint is recommenting to order the imports in several sections alphabetically: (1) python standard library (2) third party packages (3) current package
OMPython/ModelicaSystem.py:1787:16: W0612: Unused variable 'key' (unused-variable) => replace items() by values()
* remove not needed variable definitions * fix if condition for bool
python-version: ['3.10', '3.12', '3.14'] os: ['ubuntu-latest', 'windows-latest'] omc-version: ['1.25.0', 'stable', 'nightly']
5c667ae to
a11ad95
Compare
…mmand) the following classes are not changed - tehse are obsolete: - OMCSessionZMQ - OMCSessionCmd
a11ad95 to
035a549
Compare
Member
|
Rebase |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
just a reminder to remove depreciated functionality at a time ...